Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Partially fixes #496
This PR turns the warnings sent after the initial praise message into one ephemeral reply.
While this solves the issue for the warnings sent after the first praise bot response(success / failure), the initial error messages can't be made ephemeral in a neat way as we want variable behaviour from that message:
The can't change the state of an already deferred message. A potential solution can be - Not updating the deferred reply with the error and send a response like:
and then send an ephemeral followup with the error details that is private.
Would the above be a desirable solution, or should we refactor the way the bot sends responses(
might turn into a mess)?